From: Olaf Hering Date: Tue, 12 Apr 2016 13:55:19 +0000 (+0200) Subject: tools: fix compile errors with -Og X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~1261 X-Git-Url: https://dgit.raspbian.org/%22http:/www.example.com/cgi/%22https://%22%22/%22http:/www.example.com/cgi/%22https:/%22%22?a=commitdiff_plain;h=e0ec0a717d882ff0c0935b4893792d6aa95df3ef;p=xen.git tools: fix compile errors with -Og At least gcc-4.8 and older fails to recognize that err is always initialized, the build fails: xc_cpupool.c: In function 'xc_cpupool_removecpu': xc_cpupool.c:168:5: error: 'err' may be used uninitialized in this function [-Werror=maybe-uninitialized] return err; Fix this in blktap2 and libxc. Signed-off-by: Olaf Hering Cc: Ian Jackson Cc: Wei Liu Reviewed-by: Dario Faggioli Acked-by: Wei Liu Release-acked-by: Wei Liu --- diff --git a/tools/blktap2/drivers/tapdisk-vbd.c b/tools/blktap2/drivers/tapdisk-vbd.c index 89ef9edc4e..31bc2fe3d1 100644 --- a/tools/blktap2/drivers/tapdisk-vbd.c +++ b/tools/blktap2/drivers/tapdisk-vbd.c @@ -516,7 +516,7 @@ tapdisk_vbd_free_stack(td_vbd_t *vbd) int tapdisk_vbd_open_stack(td_vbd_t *vbd, uint16_t storage, td_flag_t flags) { - int i, err; + int i, err = 0; vbd->flags = flags; vbd->storage = storage; @@ -932,7 +932,7 @@ tapdisk_vbd_open_image(td_vbd_t *vbd, td_image_t *image) static int tapdisk_vbd_close_and_reopen_image(td_vbd_t *vbd, td_image_t *image) { - int i, err; + int i, err = 0; td_close(image); @@ -972,7 +972,7 @@ tapdisk_vbd_pause(td_vbd_t *vbd) int tapdisk_vbd_resume(td_vbd_t *vbd, const char *path, uint16_t drivertype) { - int i, err; + int i, err = 0; if (!td_flag_test(vbd->state, TD_VBD_PAUSED)) { EPRINTF("resume request for unpaused vbd %s\n", vbd->name); diff --git a/tools/libxc/xc_cpupool.c b/tools/libxc/xc_cpupool.c index bb99e3bed3..70011d1f34 100644 --- a/tools/libxc/xc_cpupool.c +++ b/tools/libxc/xc_cpupool.c @@ -151,7 +151,7 @@ int xc_cpupool_removecpu(xc_interface *xch, int cpu) { unsigned retries; - int err; + int err = 0; DECLARE_SYSCTL; sysctl.cmd = XEN_SYSCTL_cpupool_op;